Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

[WIP] Fix NullPointerException on pulsar 2.9.0-SNAPSHOT#718

Closed
Demogorgon314 wants to merge 5 commits into
streamnative:masterfrom
Demogorgon314:fix-null-point-pulsar-2.9.0-SNAPSHOT
Closed

[WIP] Fix NullPointerException on pulsar 2.9.0-SNAPSHOT#718
Demogorgon314 wants to merge 5 commits into
streamnative:masterfrom
Demogorgon314:fix-null-point-pulsar-2.9.0-SNAPSHOT

Conversation

@Demogorgon314

@Demogorgon314 Demogorgon314 commented Sep 8, 2021

Copy link
Copy Markdown
Member

Currently pulsar has a hangs problem, we need fix pulsar side problem then upgrade pulsar dependency.

@Demogorgon314 Demogorgon314 self-assigned this Sep 8, 2021
@Demogorgon314
Demogorgon314 marked this pull request as draft September 18, 2021 10:17
codelipenghui pushed a commit to apache/pulsar that referenced this pull request Sep 22, 2021
Link PR in KoP:
streamnative/kop#718 and [failed test unit](https://github.com/streamnative/kop/pull/718/checks?check_run_id=3541827178#step:5:353)

### Motivation
The bug cause by #11012, the `loadOrCreatePersistentTopic` has uncaught exception may cause `getTopic` hang.

### Modifications

Caught the exception

### Verifying this change

The [unit test](#11290) added by @kaushik-develop help to isolate and reproduce the issue.

### Documentation
- [x] no-need-doc 
  
This is a bug fix, no need documentation.
@Demogorgon314

Copy link
Copy Markdown
Member Author

Pulsar 2.9.0 current don't expose getZkClient(), I need find a way to use getLocalMetadataStore() instead.

@Demogorgon314
Demogorgon314 deleted the fix-null-point-pulsar-2.9.0-SNAPSHOT branch October 20, 2021 02:25
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
Link PR in KoP:
streamnative/kop#718 and [failed test unit](https://github.com/streamnative/kop/pull/718/checks?check_run_id=3541827178#step:5:353)

### Motivation
The bug cause by apache#11012, the `loadOrCreatePersistentTopic` has uncaught exception may cause `getTopic` hang.

### Modifications

Caught the exception

### Verifying this change

The [unit test](apache#11290) added by @kaushik-develop help to isolate and reproduce the issue.

### Documentation
- [x] no-need-doc 
  
This is a bug fix, no need documentation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant